Skip to content

add linter to ci, fix linter reported issues#378

Merged
AlexAndBear merged 5 commits intomainfrom
add-linter-followup
Jul 14, 2025
Merged

add linter to ci, fix linter reported issues#378
AlexAndBear merged 5 commits intomainfrom
add-linter-followup

Conversation

@AlexAndBear
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings July 14, 2025 10:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds linting to the CI pipeline and fixes numerous linter problems across documentation files. The changes focus on formatting consistency and standardizing markdown syntax.

Key Changes

  • CI Integration: Added markdown linting to the CI build pipeline
  • Syntax Standardization: Updated shell code blocks from shell to bash for consistency
  • Heading Structure: Converted various heading levels to proper markdown hierarchy (h2/h3 format)
  • Text Formatting: Removed bold formatting from paragraph text where inappropriate

Reviewed Changes

Copilot reviewed 50 out of 50 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
.woodpecker/build.yaml Adds markdown linting step to CI pipeline
.markdownlint.json Updates linter configuration to allow necessary exceptions
static/env-vars/*.md Updates shell code blocks to use bash syntax highlighting
Multiple documentation files Standardizes heading levels and removes excessive bold formatting
docs/*/README.md Updates shell commands to use bash syntax highlighting

id: migrate
title: 'Migrate'
description:: 'Guide to migrating data using rclone.'
description: 'Guide to migrating data using rclone.'
Copy link

Copilot AI Jul 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is an extra colon after 'description' that should be removed. It should be 'description:' not 'description::'.

Copilot uses AI. Check for mistakes.
@AlexAndBear AlexAndBear self-assigned this Jul 14, 2025
@github-project-automation github-project-automation bot moved this to Qualification in OpenCloud Team Board Jul 14, 2025
@AlexAndBear AlexAndBear moved this from Qualification to In Progress in OpenCloud Team Board Jul 14, 2025
AlexAndBear and others added 3 commits July 14, 2025 12:36
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ient/installation.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

@Svanvith Svanvith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AlexAndBear AlexAndBear merged commit 4951f8c into main Jul 14, 2025
2 checks passed
@AlexAndBear AlexAndBear deleted the add-linter-followup branch July 14, 2025 11:30
@github-project-automation github-project-automation bot moved this from In Progress to Done in OpenCloud Team Board Jul 14, 2025
@openclouders openclouders mentioned this pull request Jul 14, 2025
1 task
@AlexAndBear AlexAndBear changed the title add to ci, fix linter problems add linter to ci, fix linter reported issues Jul 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants